-
-
Notifications
You must be signed in to change notification settings - Fork 318
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wxGUI: Fixed E722 in ii2t_gis_set.py #4417
Conversation
Was the branch too outdated yet? |
Yeah, even though I cloned a fresh copy this morning. Normally I try to update the branch when reqd so that PR are in line with the latest version |
But if there aren't potential semantic conflicts, or not too outdated (new enforcements, similar files changed), it doesn't matter to be a couple commits out of date. Each push is about 1h of CI x 20 runners, shared between all OSGeo projects. |
Oh ok, got it. I do this to avoid any potential merge conflicts but will let it be in the future |
Fixed
E722: bare 'except'
in iit2t_gis_set.py. Added general catchallException
for cases with multiple errors. Other were mostlyOSError and shutil.Error
. Also print error message in the error notification